Skip to content

[App Menu] Fix action button popover reposition#272011

Merged
kowalczyk-krzysztof merged 2 commits into
elastic:mainfrom
kowalczyk-krzysztof:fix/app-menu-popover-button-axis
Jun 1, 2026
Merged

[App Menu] Fix action button popover reposition#272011
kowalczyk-krzysztof merged 2 commits into
elastic:mainfrom
kowalczyk-krzysztof:fix/app-menu-popover-button-axis

Conversation

@kowalczyk-krzysztof
Copy link
Copy Markdown
Member

Summary

This PR fixes a bug where for some reason, in classic mode (works fine in solution mode), the primary action item with a popover would have the popover rendered to the left instead of downRight.

Before:

Screenshot 2026-05-30 at 01 17 20

After:

Screenshot 2026-05-30 at 00 34 33

@kowalczyk-krzysztof kowalczyk-krzysztof self-assigned this May 29, 2026
@kowalczyk-krzysztof kowalczyk-krzysztof requested a review from a team as a code owner May 29, 2026 23:40
@kowalczyk-krzysztof kowalczyk-krzysztof added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// labels May 29, 2026
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/appex-sharedux (Team:SharedUX)

@kowalczyk-krzysztof kowalczyk-krzysztof enabled auto-merge (squash) May 29, 2026 23:40
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
core 150.0KB 150.1KB +98.0B
discover 1.9MB 1.9MB +98.0B
total +196.0B

History

cc @kowalczyk-krzysztof

@kowalczyk-krzysztof kowalczyk-krzysztof merged commit 535e2c1 into elastic:main Jun 1, 2026
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants